Skip to content

Conversation

@mathieulaude
Copy link

@mathieulaude mathieulaude commented Dec 30, 2025

Description

When (re)installing a VPS from Terraform, there is not equivalent of the " I do not wish to receive my VPS authentication codes by email. " option.
This PR adds an attribute do_not_send_password to disable password authentication. This allows a complete automatized workflow where the first authentication on the VPS with the SSH key is successful.

Type of change

  • Improvement (improve existing resource(s) or datasource(s))

How Has This Been Tested?

No

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings or issues
  • I have added acceptance tests that prove my fix is effective or that my feature works
  • New and existing acceptance tests pass locally with my changes
  • I ran successfully go mod vendor if I added or modify go.mod file

@mathieulaude mathieulaude requested a review from a team as a code owner December 30, 2025 17:16
@mathieulaude mathieulaude force-pushed the patch-1 branch 5 times, most recently from d89f982 to 47b0031 Compare January 11, 2026 13:29
@mathieulaude
Copy link
Author

Hi there,
I have issues running tests because of cloud_project_rancher that should not be impacted by my modification.
Can you give me some hints about how to fix this please?

> make test-compile 
go test -c $(go list ./... |grep -v 'vendor') 
cannot write test binary cloud_project_rancher.test for multiple packages:
github.com/ovh/terraform-provider-ovh/v2/ovh/datasources/cloud_project_rancher
github.com/ovh/terraform-provider-ovh/v2/ovh/resources/cloud_project_rancher
make: *** [GNUmakefile:39 : test-compile] Erreur 1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant